Refresh the docs against what N-11 actually shipped - #110
Merged
Conversation
First real staleness check on user-docs/, three days after writing it, and two pages had drifted. Both were claims about things that changed underneath them, which is exactly the failure mode the source-of-truth footers exist to catch. The architecture page said the quick-create link carries four parameters. It carries five: N-11 phase 2 added TopsInstallId, and the parent subscription now only forwards messages carrying that value while the rest land in a quarantine queue. That is a materially better story for the audience this page is written for, so it is worth saying properly rather than leaving a stale count. SECURITY.md listed three known issues, two of which are now closed -- #100 shipped and #102's template is deleted. A vulnerability policy telling researchers not to report things that are already fixed is worse than useless: it discourages reporting a regression. #101 stays, reworded to describe what is now in place rather than what was open when it was written, and #109 takes the freed slot with an honest note that it is dead permission rather than exposure. Verified rather than assumed: the IAM permissions page still holds exactly -- 21 inline policies, 3 resource-scoped, 18 unscoped, 2 managed, 2 roles -- and the findings pages' 74 rules and 4/24/32/14 severity split are unchanged. No page referenced the deleted audit template except the SECURITY.md row now removed. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
GitHub private vulnerability reporting is enabled on the repo -- verified via the API rather than assumed -- and security@teemops.com is routed. Both channels SECURITY.md names now answer, so the caveat recorded against it comes off. SECURITY.md itself needed no change: it was written naming both channels on the assumption they would exist, which is now true. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What and why
First staleness check on
user-docs/, three days after it was written and immediately after N-11 phase 2, #100 and #102 landed. Two pages had already drifted — both making claims about things that changed underneath them.This is the maintenance pass the docs conventions were built to make cheap. It found real drift on the first run.
What was stale
The architecture page said the quick-create link carries four parameters. It carries five — N-11 phase 2 added
TopsInstallId, and the parent subscription now only forwards messages carrying that value, quarantining the rest. That is a materially better story for the security reviewers this page is written for, so it now describes the filter and the quarantine queue rather than just correcting a count.SECURITY.mdlisted three known issues, two of them now closed. #100 shipped and #102's template is deleted. A vulnerability policy telling researchers not to report things that are already fixed is worse than useless — it discourages reporting a regression. #101 stays, reworded to describe what is now in place rather than what was open when it was written. #109 takes the freed slot, with an honest note that theSQS:ReceiveMessagegrant is dead permission rather than exposure.What was verified and did not need changing
Checked rather than assumed, against the current tree:
templates/iam.role.child.account.cfn.yamlapp/rules/rulesets/*.jsongrepacrossuser-docs/How it was tested
Documentation only; no application code touched.
user-docs/andSECURITY.md— all resolve.templates/iam.role.audit.account.cfn.yamlortemplates/sync.sh, both deleted by Delete the audit CFN template that defaults to the vendor's AWS account #107.Quality gate
docs/practices/developas of this commitPractices check
SECURITY.mdchange makes the known-issues list accurate, which matters more than it readsOut of scope
security@teemops.com.SECURITY.mdstill names two channels that do not answer yet — that is a repo settings change and a mail change, both yours.🤖 Generated with Claude Code